[all-commits] [llvm/llvm-project] 31c799: [cfi] Fix one -fno-sanitize-merge case, and add tw...

[Thurston Dang via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 31c7997a4acb838c94d5ab40baaf154556532ad9
      https://github.com/llvm/llvm-project/commit/31c7997a4acb838c94d5ab40baaf154556532ad9
  Author: Thurston Dang <thurston at google.com>
  Date:   2025-04-23 (Wed, 23 Apr 2025)

  Changed paths:
    M clang/lib/CodeGen/CGClass.cpp
    M clang/lib/CodeGen/CGExpr.cpp

  Log Message:
  -----------
  [cfi] Fix one -fno-sanitize-merge case, and add two TODOs (#135438)

-fno-sanitize-merge (introduced in
https://github.com/llvm/llvm-project/pull/120464) nearly works for CFI:
code that calls EmitCheck will already check the merge options. This
patch fixes one EmitTrapCheck call, which did not check the merge
options, and for two other EmitTrapChecks, adds two TODOs that explain
why it is difficult to fix them.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications