Page MenuHomePhabricator
Feed Advanced Search

Thu, Sep 9

samitolvanen added a comment to D107934: [LowerTypeTests] Emit cfi_jt aliases regardless of function export.

causes confusing stack traces in applications

Thu, Sep 9, 2:32 PM · Restricted Project, Restricted Project

Tue, Aug 31

samitolvanen added inline comments to D108479: [Clang] Add __builtin_addressof_nocfi.
Tue, Aug 31, 2:02 PM · Restricted Project
samitolvanen updated the diff for D108479: [Clang] Add __builtin_addressof_nocfi.

Fixed clang-tidy warnings, dropped an unnecessary auto.

Tue, Aug 31, 2:02 PM · Restricted Project
samitolvanen added a comment to D108478: [llvm][IR] Add no_cfi constant.

I think you might want to update the syntax files for the various text editors. See:

  • llvm/utils/vim/syntax/llvm.vim
  • llvm/utils/emacs/llvm-mode.el
  • llvm/utils/kate/llvm.xml
  • llvm/utils/vscode/llvm/syntaxes/ll.tmLanguage.yaml
Tue, Aug 31, 2:01 PM · Restricted Project
samitolvanen updated the diff for D108478: [llvm][IR] Add no_cfi constant.

Fixed clang-tidy warnings, cleaned up tests.

Tue, Aug 31, 2:00 PM · Restricted Project

Aug 20 2021

samitolvanen added reviewers for D108479: [Clang] Add __builtin_addressof_nocfi: nickdesaulniers, pcc.

Here's a PoC of the built-in function that returns the address of the function body with CFI. Based on D108478.

Aug 20 2021, 12:03 PM · Restricted Project
samitolvanen added reviewers for D108478: [llvm][IR] Add no_cfi constant: nickdesaulniers, pcc.

Here's a PoC of the "no cfi" constant type suggested by Peter. PTAL.

Aug 20 2021, 12:02 PM · Restricted Project
samitolvanen requested review of D108479: [Clang] Add __builtin_addressof_nocfi.
Aug 20 2021, 12:00 PM · Restricted Project
samitolvanen requested review of D108478: [llvm][IR] Add no_cfi constant.
Aug 20 2021, 11:58 AM · Restricted Project

Aug 18 2021

samitolvanen added a comment to D105265: [X86] AVX512FP16 instructions enabling 3/6.
Aug 18 2021, 10:59 AM · Restricted Project, Restricted Project

Aug 12 2021

samitolvanen added a comment to D107934: [LowerTypeTests] Emit cfi_jt aliases regardless of function export.

Thanks, Nick. I tested a kernel build with both ThinLTO and regular LTO + CFI, and can confirm that with this patch vmlinux has the previously missing .cfi_jt symbols again.

Aug 12 2021, 9:40 AM · Restricted Project, Restricted Project

Aug 3 2021

samitolvanen committed rG7ce1c4da7726: ThinLTO: Fix inline assembly references to static functions with CFI (authored by samitolvanen).
ThinLTO: Fix inline assembly references to static functions with CFI
Aug 3 2021, 11:48 AM
samitolvanen closed D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Aug 3 2021, 11:48 AM · Restricted Project, Restricted Project

Jul 30 2021

samitolvanen added inline comments to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 30 2021, 12:20 PM · Restricted Project, Restricted Project
samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Also skip functions with names incompatible with XCOFF.

Jul 30 2021, 11:39 AM · Restricted Project, Restricted Project
samitolvanen added inline comments to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 30 2021, 11:21 AM · Restricted Project, Restricted Project

Jul 21 2021

samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

As we only care about fixing inline assembly references, mangled names are
not that important in the first place. This version skips any functions
that have unusual characters in their names that would otherwise require
quotes, which includes any functions with MSVC compatible name mangling.

Jul 21 2021, 3:17 PM · Restricted Project, Restricted Project

Jul 20 2021

samitolvanen planned changes to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 20 2021, 4:59 PM · Restricted Project, Restricted Project
samitolvanen added inline comments to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 20 2021, 4:32 PM · Restricted Project, Restricted Project
samitolvanen added inline comments to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 20 2021, 3:41 PM · Restricted Project, Restricted Project
samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Folded in D106392.

Jul 20 2021, 2:08 PM · Restricted Project, Restricted Project
samitolvanen reopened D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 20 2021, 2:07 PM · Restricted Project, Restricted Project
samitolvanen abandoned D106392: ThinLTO: Fix promotion aliases with -msvc targets.

Nick suggested reverting until we figure out if this is the ideal fix here. I'll fold this to the original patch.

Jul 20 2021, 2:03 PM · Restricted Project
samitolvanen added a reverting change for D104058: ThinLTO: Fix inline assembly references to static functions with CFI: rGe901e581ef45: Revert "ThinLTO: Fix inline assembly references to static functions with CFI".
Jul 20 2021, 2:00 PM · Restricted Project, Restricted Project
samitolvanen added a reverting change for rG700d07f8ce6f: ThinLTO: Fix inline assembly references to static functions with CFI: rGe901e581ef45: Revert "ThinLTO: Fix inline assembly references to static functions with CFI".
Jul 20 2021, 2:00 PM
samitolvanen committed rGe901e581ef45: Revert "ThinLTO: Fix inline assembly references to static functions with CFI" (authored by samitolvanen).
Revert "ThinLTO: Fix inline assembly references to static functions with CFI"
Jul 20 2021, 2:00 PM
samitolvanen retitled D106392: ThinLTO: Fix promotion aliases with -msvc targets from ThinLTO: Fix promotion aliases with MSVC name mangling to ThinLTO: Fix promotion aliases with -msvc targets.
Jul 20 2021, 1:45 PM · Restricted Project
samitolvanen added a comment to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

This patch broke the sanitizer-windows bot: https://lab.llvm.org/buildbot/#/builders/127/builds/14257

Failed Tests (4):
  cfi-devirt-lld-thinlto-x86_64 :: anon-namespace.cpp
  cfi-devirt-lld-thinlto-x86_64 :: simple-pass.cpp
  cfi-standalone-lld-thinlto-x86_64 :: anon-namespace.cpp
  cfi-standalone-lld-thinlto-x86_64 :: simple-pass.cpp

Please revert or fix.

Jul 20 2021, 1:40 PM · Restricted Project, Restricted Project
samitolvanen added reviewers for D106392: ThinLTO: Fix promotion aliases with -msvc targets: nickdesaulniers, pcc.
Jul 20 2021, 1:13 PM · Restricted Project
samitolvanen requested review of D106392: ThinLTO: Fix promotion aliases with -msvc targets.
Jul 20 2021, 1:12 PM · Restricted Project
samitolvanen committed rG700d07f8ce6f: ThinLTO: Fix inline assembly references to static functions with CFI (authored by samitolvanen).
ThinLTO: Fix inline assembly references to static functions with CFI
Jul 20 2021, 10:30 AM
samitolvanen closed D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 20 2021, 10:30 AM · Restricted Project, Restricted Project

Jul 16 2021

samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Added REQUIRES: x86-registered-target to tests.

Jul 16 2021, 3:27 PM · Restricted Project, Restricted Project
samitolvanen reopened D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

The tests that now specify a target triple also need ; REQUIRES: x86-registered-target or they will obviously fail. I'll upload another revision.

Jul 16 2021, 2:55 PM · Restricted Project, Restricted Project
samitolvanen added a reverting change for rG8e3b5cb39eef: ThinLTO: Fix inline assembly references to static functions with CFI: rG0ad1d9fdf22d: Revert "ThinLTO: Fix inline assembly references to static functions with CFI".
Jul 16 2021, 2:48 PM
samitolvanen committed rG0ad1d9fdf22d: Revert "ThinLTO: Fix inline assembly references to static functions with CFI" (authored by samitolvanen).
Revert "ThinLTO: Fix inline assembly references to static functions with CFI"
Jul 16 2021, 2:48 PM
samitolvanen added a reverting change for D104058: ThinLTO: Fix inline assembly references to static functions with CFI: rG0ad1d9fdf22d: Revert "ThinLTO: Fix inline assembly references to static functions with CFI".
Jul 16 2021, 2:48 PM · Restricted Project, Restricted Project
samitolvanen committed rG8e3b5cb39eef: ThinLTO: Fix inline assembly references to static functions with CFI (authored by samitolvanen).
ThinLTO: Fix inline assembly references to static functions with CFI
Jul 16 2021, 2:34 PM
samitolvanen closed D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 16 2021, 2:34 PM · Restricted Project, Restricted Project
samitolvanen added a comment to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Change LGTM, but I don't understand why the following tests are modified:

  • llvm/test/ThinLTO/X86/devirt2.ll

This is needed to fix two missing symbol resolution errors that are caused by the aliases we added.

I'm curious if this will lead to breakages with LTO in general? I suppose not, since it's llvm-lto2 that needs the explicit list of symbols that can be linked against.

Jul 16 2021, 10:59 AM · Restricted Project, Restricted Project

Jul 14 2021

samitolvanen added a comment to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Change LGTM, but I don't understand why the following tests are modified:

  • llvm/test/ThinLTO/X86/devirt2.ll
Jul 14 2021, 1:18 PM · Restricted Project, Restricted Project

Jul 13 2021

samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Moved the alias creation to module level inline assembly to avoid issues with LowerTypeTestsModule, based on pcc's suggestion.

Jul 13 2021, 11:21 AM · Restricted Project, Restricted Project
samitolvanen reopened D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jul 13 2021, 11:19 AM · Restricted Project, Restricted Project

Jun 23 2021

samitolvanen added a comment to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Thanks for the revert, I'll take a look.

Jun 23 2021, 8:12 PM · Restricted Project, Restricted Project
samitolvanen committed rGe3d24b45b8f8: ThinLTO: Fix inline assembly references to static functions with CFI (authored by samitolvanen).
ThinLTO: Fix inline assembly references to static functions with CFI
Jun 23 2021, 11:10 AM
samitolvanen closed D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jun 23 2021, 11:10 AM · Restricted Project, Restricted Project
samitolvanen added inline comments to D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jun 23 2021, 8:24 AM · Restricted Project, Restricted Project

Jun 22 2021

samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Fix a use-of-uninitialized-value error.

Jun 22 2021, 2:35 PM · Restricted Project, Restricted Project
samitolvanen reopened D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jun 22 2021, 2:34 PM · Restricted Project, Restricted Project
samitolvanen added a reverting change for rG4474958d3a97: ThinLTO: Fix inline assembly references to static functions with CFI: rG33c9438f1166: Revert "ThinLTO: Fix inline assembly references to static functions with CFI".
Jun 22 2021, 12:11 PM
samitolvanen committed rG33c9438f1166: Revert "ThinLTO: Fix inline assembly references to static functions with CFI" (authored by samitolvanen).
Revert "ThinLTO: Fix inline assembly references to static functions with CFI"
Jun 22 2021, 12:11 PM
samitolvanen added a reverting change for D104058: ThinLTO: Fix inline assembly references to static functions with CFI: rG33c9438f1166: Revert "ThinLTO: Fix inline assembly references to static functions with CFI".
Jun 22 2021, 12:11 PM · Restricted Project, Restricted Project
samitolvanen committed rG4474958d3a97: ThinLTO: Fix inline assembly references to static functions with CFI (authored by samitolvanen).
ThinLTO: Fix inline assembly references to static functions with CFI
Jun 22 2021, 10:02 AM
samitolvanen closed D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jun 22 2021, 10:02 AM · Restricted Project, Restricted Project

Jun 17 2021

samitolvanen updated the diff for D104058: ThinLTO: Fix inline assembly references to static functions with CFI.

Moved the test to llvm/test/Transforms/ThinLTOBitcodeWriter.

Jun 17 2021, 12:49 PM · Restricted Project, Restricted Project

Jun 10 2021

samitolvanen added reviewers for D104058: ThinLTO: Fix inline assembly references to static functions with CFI: nickdesaulniers, pcc, tejohnson, kees, eugenis.
Jun 10 2021, 1:47 PM · Restricted Project, Restricted Project
samitolvanen requested review of D104058: ThinLTO: Fix inline assembly references to static functions with CFI.
Jun 10 2021, 1:44 PM · Restricted Project, Restricted Project
samitolvanen updated the diff for D104043: [IR] Value: Fix OpCode checks.

Changed the message to refer to CallBase instead of CallInst.

Jun 10 2021, 12:10 PM · Restricted Project
samitolvanen added a comment to D104043: [IR] Value: Fix OpCode checks.

Thanks, Nick. I don't have commit access, so would you mind committing the patch? I confirmed that check-all passes for me with the patch applied.

Jun 10 2021, 11:51 AM · Restricted Project
samitolvanen added reviewers for D104043: [IR] Value: Fix OpCode checks: nickdesaulniers, void.
Jun 10 2021, 9:29 AM · Restricted Project
samitolvanen requested review of D104043: [IR] Value: Fix OpCode checks.
Jun 10 2021, 9:29 AM · Restricted Project

Jun 8 2021

samitolvanen added a comment to D103120: LTO: Export functions referenced by non-canonical CFI jump tables.

Thanks, Peter. I don't have commit access, so Nick, would you mind committing this for me? The Windows build failure looks unrelated.

Jun 8 2021, 3:05 PM · Restricted Project

Jun 2 2021

samitolvanen added inline comments to D103120: LTO: Export functions referenced by non-canonical CFI jump tables.
Jun 2 2021, 8:14 AM · Restricted Project
samitolvanen added a comment to D103120: LTO: Export functions referenced by non-canonical CFI jump tables.

Is this related to b/158034873 or another reported bug?

Jun 2 2021, 8:08 AM · Restricted Project

May 25 2021

samitolvanen added reviewers for D103120: LTO: Export functions referenced by non-canonical CFI jump tables: pcc, nickdesaulniers.
May 25 2021, 3:24 PM · Restricted Project
samitolvanen requested review of D103120: LTO: Export functions referenced by non-canonical CFI jump tables.
May 25 2021, 3:22 PM · Restricted Project