Opened 9 years ago
Last modified 9 years ago
#46754 closed defect
llvm-3.* does not recognize -assertions — at Version 2
| Reported by: | pguyot (Paul Guyot) | Owned by: | jeremyhu (Jeremy Huddleston Sequoia) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.3.3 |
| Keywords: | Cc: | ||
| Port: | llvm-3.3 llvm-3.4 llvm-3.5 llvm-3.6 llvm-3.7 |
Description (last modified by pguyot (Paul Guyot))
Even when -assertions is passed, assertion mode is ON.
I believe --enable-assertions is NOP and --disable-assertions should be passed instead as assertions are enabled by default.
This is also the case with at least 3.7 and 3.5 and maybe other versions as well.
./configure --help indeed prints:
--enable-assertions Compile with assertion checks enabled (default is
YES)
A patch is attached that fixes this for llvm-3.6. I didn't bump revision, though. Yet, bumping revision is required at least for llvm-3.5 as the -assertions variant is available as a binary package (but with assertion-mode as ON).
Change History (3)
Changed 9 years ago by pguyot (Paul Guyot)
| Attachment: | llvm-assertions.diff added |
|---|
comment:1 Changed 9 years ago by pguyot (Paul Guyot)
| Description: | modified (diff) |
|---|---|
| Summary: | llvm-3.6 does not recognize -assertions → llvm-3.* does not recognize -assertions |
comment:2 Changed 9 years ago by pguyot (Paul Guyot)
| Description: | modified (diff) |
|---|---|
| Port: | llvm-3.5 llvm-3.6 llvm-3.7 added; llvm removed |
Note: See
TracTickets for help on using
tickets.