This revision was automatically updated to reflect the committed changes.
Closed by commit rG9afc57dcb2e5: [CodeGen][UBSan] Handle sugared QualTypes
correctly in (authored by usama54321).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157445/new/
usama54321 updated this revision to Diff 550477.
usama54321 added a comment.
Updated commit message
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157445/new/
https://reviews.llvm.org/D157445
Files:
clang/lib/CodeGen/CodeGenFunction.cpp
usama54321 updated this revision to Diff 550475.
usama54321 retitled this revision from "[CodeGen][UBSan] Add support for
handling attributed functions in getUBSanFunctionTypeHash." to
"[CodeGen][UBSan] Add support for handling attributed functions in
getUBSanFunctionTypeHash.".
usama54321
usama54321 created this revision.
Herald added subscribers: manas, steakhal, ASDenysPetrov, martong, dkrupp,
donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware,
xazax.hun.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added a
usama54321 updated this revision to Diff 430469.
usama54321 added a comment.
Fixed clang-format issue
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125840/new/
https://reviews.llvm.org/D125840
Files:
usama54321 updated this revision to Diff 430450.
usama54321 added a comment.
Updated tests
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125840/new/
https://reviews.llvm.org/D125840
Files:
clang/lib/StaticAnalyzer/Checkers/NSErrorChecker.cpp
usama54321 updated this revision to Diff 431516.
usama54321 added a comment.
- updated canResolveToExpr to accept both statements and expressions. Removed
unnecessary code
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126034/new/
usama54321 updated this revision to Diff 431490.
usama54321 added a comment.
Added a separate check for unevaluated statements. Updated InfiniteLoopCheck to
use new check
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126034/new/
This revision was automatically updated to reflect the committed changes.
usama54321 marked an inline comment as done.
Closed by commit rG602682225ad6: bugfix in InfiniteLoopCheck to not print
warnings for unevaluated loops (authored by usama54321).
Changed prior to commit:
usama54321 created this revision.
Herald added a subscriber: carlosgalvezp.
Herald added a project: All.
This revision was not accepted when it landed; it landed in state "Draft".
This revision was automatically updated to reflect the committed changes.
Closed by commit rG63ecb7dcc80d: bugfix in
usama54321 updated this revision to Diff 431497.
usama54321 added a comment.
Updating patch
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126034/new/
https://reviews.llvm.org/D126034
Files:
usama54321 created this revision.
Herald added a subscriber: carlosgalvezp.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added projects: clang, clang-tools-extra.
Herald added a subscriber: cfe-commits.
Repository:
rG LLVM Github Monorepo
usama54321 created this revision.
Herald added a subscriber: carlosgalvezp.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added projects: clang, clang-tools-extra.
Herald added a subscriber: cfe-commits.
Repository:
rG LLVM Github Monorepo
usama54321 updated this revision to Diff 488408.
usama54321 added a comment.
Addressing comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141550/new/
https://reviews.llvm.org/D141550
Files:
usama54321 marked 3 inline comments as done.
usama54321 added inline comments.
Comment at: compiler-rt/lib/ubsan/CMakeLists.txt:118
+if (NOT APPLE)
+ add_compiler_rt_runtime(clang_rt.ubsan
+STATIC
delcypher wrote:
> I think you may have
usama54321 marked an inline comment as done.
usama54321 added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:219
+def err_drv_unsupported_static_ubsan_darwin : Error<
+ "Static UBSan runtime is not supported on darwin">;
def
usama54321 created this revision.
Herald added a subscriber: Enna1.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added subscribers: Sanitizers, cfe-commits, MaskRay.
Herald added projects: clang, Sanitizers.
This patch removes the static ubsan runtime on Apple
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGa44477b1f4b5: [CompilerRT] Remove ubsan static runtime on
Apple (authored by usama54321).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST
usama54321 created this revision.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added subscribers: cfe-commits, MaskRay.
Herald added a project: clang.
rdar://99200922
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D142421
Files:
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG5b6dbdecba0b: [CodeGen] bugfix: ApplyDebugLocation goes out
of scope before intended (authored by usama54321).
Repository:
rG LLVM Github
usama54321 created this revision.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
rdar://103570533
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D142243
Files:
usama54321 created this revision.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
rdar://103570533
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D142242
Files:
usama54321 updated this revision to Diff 490922.
usama54321 added a comment.
Renaming identifiers in test
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142242/new/
https://reviews.llvm.org/D142242
Files:
clang/lib/CodeGen/CodeGenFunction.cpp
usama54321 updated this revision to Diff 490923.
usama54321 added a comment.
Renaming identifiers in test
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142242/new/
https://reviews.llvm.org/D142242
Files:
clang/lib/CodeGen/CodeGenFunction.cpp
usama54321 updated this revision to Diff 490949.
usama54321 added a comment.
Removing unnecessary code from test
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142243/new/
https://reviews.llvm.org/D142243
Files:
usama54321 added a comment.
The libcxx test failure seems unrelated and seems to be failing on master
https://buildkite.com/llvm-project/libcxx-ci/builds?branch=master
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142243/new/
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG57690a8ece9e: [Sanitizers] fix -fno-sanitize-link-runtime
for darwin (authored by usama54321).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE
usama54321 added a comment.
I have reverted the commit for now
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144672/new/
https://reviews.llvm.org/D144672
___
cfe-commits mailing list
usama54321 added inline comments.
Comment at:
compiler-rt/test/asan/TestCases/replaceable_new_delete_static.cpp:10-11
+
+// darwin only supports shared-libsan, so this should fail.
+// XFAIL: darwin
+
yln wrote:
> zixuw wrote:
> > dmaclach wrote:
> > > yln
This revision was automatically updated to reflect the committed changes.
Closed by commit rG665e32ee0ffd: [Sanitizers] Error out for -static-libsan on
darwin (authored by usama54321).
Changed prior to commit:
https://reviews.llvm.org/D144672?vs=502474=502569#toc
Repository:
rG LLVM Github
usama54321 added inline comments.
Comment at:
compiler-rt/test/asan/TestCases/replaceable_new_delete_static.cpp:10-11
+
+// darwin only supports shared-libsan, so this should fail.
+// XFAIL: darwin
+
yln wrote:
> usama54321 wrote:
> > yln wrote:
> > > zixuw
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG4e7d40e0928c: [Sanitizers] Error out for -static-libsan on
darwin (authored by usama54321).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE
usama54321 accepted this revision.
usama54321 added a comment.
This revision is now accepted and ready to land.
I was verifying that static versions of these libraries are not present on
darwin. Sorry for the delay.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
usama54321 reopened this revision.
usama54321 added a comment.
This revision is now accepted and ready to land.
@dmaclach Can you please take a look and fix this? Thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144672/new/
usama54321 created this revision.
Herald added a project: All.
usama54321 requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
rdar://113144087
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D157445
Files:
usama54321 updated this revision to Diff 548723.
usama54321 added a comment.
Fix commit message and description, and added comment explaining the test.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157445/new/
https://reviews.llvm.org/D157445
36 matches
Mail list logo