[Verifier] Verify attributes for oracle-function in visitFunctionAttrs (#242063)
The behavior before this patch seems to be a historical artifact.
Function attributes/signature are a property of the function and should
be validated there rather than during their use inside of
llvm.speculative.load.
Assisted-By: LLM
[SPARC] Keep the %o7 restore in the delay slot of leaf tail calls (#242281)
The return address is restored in the delay slot of a tail call in a
leaf function. With the post-RA scheduler enabled another instruction
might be moved into that slot, and the restore could happen before the
call which causes %o7 to get overwritten. To prevent that, mark it as
FrameDestroy, and check for it in isSchedulingBoundary().
Assisted-by: Claude
[libc] Make alloca visible from stdlib.h (#241695)
Let `<stdlib.h>` also declare `alloca`. Some code assumes that it'll
be exposed through `<stdlib.h>` header, plus `<alloca.h>` is not
a standard header anyway and is missing on some platforms (e.g.
on BSD systems). Use YAML fragments, similar to what we do
for non-standard `<malloc.h>` header on GNU systems.
AI disclosure: this patch was written with Claude Code. I reviewed the
change and built it in a full build, and the include and header tests
pass
Assisted-by: Claude
[Clang][analyzer][NFC] Re-enable const variable redeclaration test (#241705)
`Analysis/redecl.c` already produces the expected `TRUE` result, but its
`-verify` run fails on an unannotated `'extern' variable has an
initializer` warning. Add the missing `expected-warning` and remove
`XFAIL: *`, retaining the original input and the existing `TRUE` check.
The test was added by 850361f6c1db6bd8a85b65a8c56e0e097eaa0635 (D46421,
April 23, 2019), which also changed the constant-variable lookup in
`RegionStoreManager::getBindingForVar` from `getInit()` to
`getAnyInitializer()`. Native X86 Release builds with assertions enabled
confirm the behavior change at that commit: its parent
330bc8af131c5b42efe120dbf70f47881d02d3a3 reports both `FALSE` and
`TRUE`, while the commit reports only `TRUE`. Both still emit the
extern-initializer warning, so the original test remains XFAIL even
after the analysis result is corrected.
This change annotates the warning and removes the XFAIL marker, allowing
the test to run normally again.
Prepared with assistance from Codex.
[libc] Have sys/time.h import certain symbols from sys/select.h (#241696)
Per https://pubs.opengroup.org/onlinepubs/9799919799/basedefs/sys_time.h.html,
`sys/time.h` should provide certain macro, types, and functions
that are also declared in `sys/select.h`. Use existing YAML fragment
machinery to support this.
Some programs like procps relies on it.
AI disclosure: this patch was written with Claude Code. I reviewed the
change and built it in a full build, and the include and header tests
pass
Assisted-by: Claude
[SPARC][Driver] Add -m(no-)app-regs flags (#242193)
Add GCC's -mapp-regs and -mno-app-regs. -mno-app-regs reserves the
application registers G2, G3 and G4, the same as passing -ffixed-g2
-ffixed-g3 -ffixed-g4. -mapp-regs is the default.
Fixes #18525
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
[libc] Have netdb.h carry netinet/in.h (#241697)
See https://pubs.opengroup.org/onlinepubs/9799919799/ - which
states that `Inclusion of the <netdb.h> header may also make
visible all symbols from <netinet/in.h>`.
AI disclosure: this patch was written with Claude Code. I reviewed the
change and built it in a full build, and the include and header tests
pass
Assisted-by: Claude
IR: Answer mayHaveSideEffects with one switch
Previously this emitted 3 calls, each switching over the opcodes. Fold
to one which only needs to query attributes for calls.
CTMark instructions:u: O3 -0.13%, ThinLTO -0.17%.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
netinet: Maintain RTF_LOCAL in the kernel
* Expose in_rtlocalize() for using in route_output_change_callback() to
correctly set RTF_LOCAL, taking into account of the IFA/IFP changes.
* Add RTF_LOCAL to PRESERVED_RTF and ignore the RTF_LOCAL flag from
userspace, making this flag be maintained by the kernel.
* Clean up arp_rtrequest() to directly check the RTF_LOCAL flag instead
of comparing the IP addresses.
net: Normalize IPv4 /32 route to a host route (RTF_HOST)
An IPv4 /32 route denotes a host route (RTF_HOST). Add the
rt_hostroute_normalize() helper to normalize such routes by setting the
RTF_HOST flag. Use the helper in both rtrequest1() and
rtsearch_msghandler() so that this normalization applies to RTM_ADD,
RTM_GET, RTM_CHANGE and RTM_LOCK.
netinet: Move loopback route maintain functions to in.c
The ifa_{add,del}_loopback_route() functions are only used in
netinet/in.c, so just move them to there and make them static.
Meanwhile, rename them to in_{add,del}_loopback_route() for consistency.
gmp: Clean up makefile
* Use per-target flags "FLAGS.<target>" to replace the FLAGS_GROUPS,
expressing the intention/requirement more clearly.
* Introduce helper varaibles to clean up the loop code.
* Remove the unused "mpn/generic" directory.
netinet: Clean up interface routes handling in arp_rtrequest()
in_addroute() gives every non-host, non-multicast IPv4 route either
RTF_CLONING or RTF_PRCLONING, so manually added interface routes would
have RTF_PRCLONING. As a result, we can clean up arp_rtrequest() to
check either RTF_CLONING or RTF_PRCLONING in handling a cloned route,
without hacking the RTF_CLONING flag.
An edge case is a non-host route with a "/32" mask. However, that case
doesn't make practical sense and should be normalized to a host route.
That will be addressed in the coming commits.
net: Handle R_Malloc() failure in rt_setshims()
Introduce the rt_freeshims() helper to free the memory allocated by
rt_setshims(), and also call it in rtrequest1() to avoid potential
memory leak.
This only applies MPLS.
netinet: Improve local routes handling in in_addroute()
* Extract the in_rtlocalize() helper function to improve the handling of
local routes, including explicit loopback routes added by
ifa_add_loopback_route().
* Skip modifying IFA/IFP for the explicit loopback routes, and thus
remove the pre-setting RTF_LOCAL from ifa_maintain_loopback_route().
No functional changes.
[InstCombine] Simplify log(exp10(y)) to y * log(10) (#239581)
Updated `LibCallSimplifier::optimizeLog` to recognize the `llvm.exp10`
intrinsic while folding:
```
log(exp10(y)) -> y * log(10)
```
Changes:
- Updated the if statement to check the arguments for the intrinsic and
removed the "TODO" comment at the top of the statement
- Added two tests in `llvm/test/Transforms/InstCombine/log-pow.ll`:
* log_exp10_intrinsic
* log_exp10_intrinsic_not_fast (a negative test to ensure the transform
works only if the exp10 intrinsic has the fast flag)
Closes #226865
[llvm-strings] Call xcrun instead of relying on preset variables (#242580)
Suggested by llvm-beanz as some CMake setups will not pre-set those
variables.
DAG: Look through fp_extend/fp_round when folding bitcast of fcopysign
visitFCOPYSIGN strips these from the sign operand, but the bitcast may be
visited first, leaving the conversion live in the integer form. Strip it
in the fold too, so the result does not depend on visitation order.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
DAG: Use shl of any_extend to place the sign bit when folding bitcast of fcopysign
When the sign operand is narrower than the result, only its sign bit is
used. Move it into place with a shift of an any_extend instead of a
sign_extend.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
auditdistd, bhyve: drop OPENSSL_API_COMPAT
Neither uses any OpenSSL function deprecated in 3.0 or later. Both build
with no deprecation warnings against OpenSSL 3.5 and 4.1 without it.
